ci: Bootstrap the test server on PHP 8.3 - #6563
Merged
Merged
Conversation
Every Garm run against server master fails in "configure server":
This version of Nextcloud requires at least PHP 8.3
You are currently running 8.2.31. Please update your PHP version.
Nextcloud master raised its minimum to PHP 8.3, so the shallow-server image
pinned at php8.2 can no longer install it and the job dies before the emulator
is ever started.
Point the non-stable22 matrix entry at the php8.3 image. Note the tag differs:
the php8.2 image publishes ":1" while php8.3 currently only publishes ":latest",
so this is not a straight substitution of the version in the tag. That does mean
the pin is looser than before and should be tightened once a numbered tag is
published.
The stable22 branch of the expression keeps php8.0. It is unreachable today,
since the matrix only lists master, but it belongs with that server version if
the entry is ever restored.
Assisted-by: Claude Code:claude-opus-5
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
Contributor
|
APK file: https://github.com/nextcloud/talk-android/actions/runs/32754661739/artifacts/9531134545 |
Member
Author
|
/backport to stable-25.0.x |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The problem
Every Garm run against server
masterfails inconfigure server, before the emulator is everstarted:
Nextcloud master raised its minimum to PHP 8.3, so the shallow-server image pinned at
php8.2can nolonger install it.
Example failure: https://github.com/nextcloud/talk-android/actions/runs/32749478902/job/97502761100
The fix
Point the non-
stable22matrix entry at thephp8.3image.Note the tag differs. This is not a straight substitution of the version inside the tag — the two
images do not publish the same tags:
continuous-integration-shallow-server-php8.21continuous-integration-shallow-server-php8.3latestcontinuous-integration-shallow-server-php8.4So
php8.3:1would have 404'd. The pin is therefore looser than it was and should be tightened oncea numbered tag is published upstream.
The
stable22branch of the expression keepsphp8.0. It is unreachable today, since the matrixonly lists
master, but it belongs with that server version if the entry is ever restored.Still open
The same job log shows a second error immediately above the PHP one:
That originates in
initnc.shinside the container rather than in this repo, and the job failed onthe PHP check regardless, so it is untouched here. This PR's own CI run should show whether it is
benign or a second problem that only surfaces once bootstrapping gets further.
AI tool use
The change was prepared with Claude Code, which read the failing job log, queried the GHCR registry
to establish which image tags actually exist, and made the one-line edit. Reviewed and signed off by
me. Commit carries an
Assisted-by:trailer.🤖 Generated with Claude Code